Would it make sense to also add an onDestroy to the TemplatePortalDirective as well (detach itself from the host on destroy)

LGTM

@andrewseguin We don't want to have the portal detach itself because the template may be rendered in a host somewhere completely separated from where the portal originally came from.

However, we should also make this change to DomPortalHost
(but wait for #1704 to go in before doing that)

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}